Skip to content

✅ Simplify password reset E2E tests - #2453

Merged
alejsdev merged 1 commit into
masterfrom
ref/email-handling
Aug 21, 2026
Merged

✅ Simplify password reset E2E tests#2453
alejsdev merged 1 commit into
masterfrom
ref/email-handling

Conversation

@alejsdev

@alejsdev alejsdev commented Aug 21, 2026

Copy link
Copy Markdown
Member

Pull Request

✅ Simplify password reset E2E tests

Description

The last email abstraction is unnecesary now, so we just poll /api/v1/search for the matching recipient, take the first matching message ID, fetch its HTML from /view/{ID}.html and assert against that HTML

AI Disclaimer

Reviewed with opus 5

Checklist

  • This PR links to a GitHub Discussion for the proposed code change.
  • I added tests for the change.
  • The new or updated tests fail on the main branch and pass on this PR.
  • Coverage stays at 100%.
  • The documentation explains the change if needed.

@alejsdev
alejsdev merged commit 7c11b64 into master Aug 21, 2026
27 of 29 checks passed
@alejsdev
alejsdev deleted the ref/email-handling branch August 21, 2026 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants